feat(cli): remove server functionality (now in @finos/calm-server package)#2195
Merged
rocketstack-matt merged 3 commits intofinos:mainfrom Mar 2, 2026
Merged
Conversation
markscott-ms
force-pushed
the
issues/2051-delete-cli-server
branch
from
bb187d0 to
f2d4e05
Compare
Contributor
There was a problem hiding this comment.
Pull request overview
This PR removes the experimental HTTP server capability from the @finos/calm-cli workspace now that the functionality has moved to the standalone @finos/calm-server package, simplifying the CLI surface area and related maintenance.
Changes:
- Removed the
servercommand from the CLI entrypoint and deleted all CLI server implementation code undercli/src/server/. - Removed server-related tests (unit, route, and e2e) and cleaned up server-specific dependencies.
- Updated CLI documentation/help excerpts to no longer mention the
servercommand.
Reviewed changes
Copilot reviewed 16 out of 17 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| package-lock.json | Removes server-related packages from the CLI workspace; also includes additional lockfile metadata churn. |
| docs/docs/working-with-calm/using-the-cli.md | Removes server from the documented CLI command list. |
| cli/src/server/routes/validation-route.ts | Deletes validation HTTP route implementation (moved out of CLI). |
| cli/src/server/routes/validation-route.spec.ts | Deletes route tests that depended on supertest. |
| cli/src/server/routes/routes.ts | Deletes server route wiring. |
| cli/src/server/routes/routes.spec.ts | Deletes server route wiring unit tests. |
| cli/src/server/routes/health-route.ts | Deletes /health route implementation. |
| cli/src/server/routes/health-route.spec.ts | Deletes /health route tests. |
| cli/src/server/cli-server.ts | Deletes express server startup helper. |
| cli/src/server/cli-server.integration.spec.ts | Deletes server startup integration test. |
| cli/src/cli.ts | Removes the server command registration/options from the CLI. |
| cli/src/cli.spec.ts | Removes unit test coverage for the CLI server command wiring. |
| cli/src/cli.e2e.spec.ts | Removes e2e coverage for calm server behavior. |
| cli/package.json | Removes server-related dependencies (express-rate-limit, supertest, @types/supertest). |
| cli/README.md | Removes server from the CLI help text and deletes the server documentation section. |
| cli/AGENTS.md | Removes server from the “Key Commands” / directory overview docs for AI assistants. |
| calm-ai/tools/calm-cli-instructions.md | Removes references to server from CLI instructions used by AI tooling. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
markscott-ms
requested review from
a team,
LeighFinegold,
Thels,
aidanm3341,
grahampacker-ms,
jpgough-ms,
lbulanti-ms,
rocketstack-matt and
willosborne
as code owners
rocketstack-matt
approved these changes
Mar 2, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
To be merged only after #2194 and #2199 , and
@finos/calm-serverpackage has been successfully published.This pull request removes the experimental HTTP server ("server" command) functionality from the CALM CLI, along with all related code, tests, documentation, and dependencies. The CLI and documentation are now focused solely on the core commands (
generate,validate,template,docify,init-ai).The validation server had been documented in the CALM website "Working with CALM" section.
Whilst this is a BREAKING CHANGE, it is removal of an Experimental feature and does not merit a major CLI release version change.
Functionality has been moved to a separate
@finos/calm-serverpackage.Key changes include:
CLI Command and Documentation
servercommand from CLI documentation files (cli/README.md,calm-ai/tools/calm-cli-instructions.md,cli/AGENTS.md), including usage instructions, options, and examples. [1] [2] [3] [4] [5]cli/src/cli.ts) to remove theservercommand and its options. [1] [2]Codebase Simplification
cli/src/server/cli-server.ts,cli/src/server/routes/routes.ts,cli/src/server/routes/health-route.ts). [1] [2] [3]cli/AGENTS.md).Test Suite Cleanup
cli/src/cli.e2e.spec.ts,cli/src/cli.spec.ts,cli/src/server/cli-server.integration.spec.ts,cli/src/server/routes/health-route.spec.ts,cli/src/server/routes/routes.spec.ts). [1] [2] [3] [4] [5] [6] [7]Dependency Removal
express-rate-limit,supertest,@types/supertest) fromcli/package.json. [1] [2]Type of Change
Affected Components
cli/)calm/)calm-ai/)calm-hub/)calm-hub-ui/)calm-server/)calm-widgets/)docs/)shared/)calm-plugins/vscode/)Commit Message Format ✅
Testing
Checklist